Skip to content

CLOUDP-311380: release cleanup #741

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 4 commits into from
Jun 6, 2025
Merged

CLOUDP-311380: release cleanup #741

merged 4 commits into from
Jun 6, 2025

Conversation

@andreaangiolillo
Copy link
Collaborator

@andreaangiolillo andreaangiolillo commented Jun 4, 2025 •
edited
Loading

Proposed changes

Jira ticket: CLOUDP-311380

This PR add an additional steps to the foas release process to delete any upcoming api specifications if the upcoming version is no longer listed in the version.json file

Test

I tested my changes in a fork:

  1. 43543497225: this run does not delete any file since the 2025-09-22.upcoming is not released to the stable 2025-09-22
  2. 15469687556: here is an example of the PR that the cleanup workflow opened -> APIx-Bot: Release Cleanup 🧹🍃 andreaangiolillo/openapi-test#60

@andreaangiolillo andreaangiolillo changed the title CLOUDP-311380: release cleanup CLOUDP-311380: release cleanup :brush Jun 5, 2025
@andreaangiolillo andreaangiolillo changed the title CLOUDP-311380: release cleanup :brush CLOUDP-311380: release cleanup Jun 5, 2025
@andreaangiolillo andreaangiolillo marked this pull request as ready for review June 5, 2025 14:36
@andreaangiolillo andreaangiolillo requested a review from a team as a code owner June 5, 2025 14:36
cveticm
cveticm approved these changes Jun 5, 2025
View reviewed changes
Copy link
Collaborator

@cveticm cveticm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just one nit

@andreaangiolillo andreaangiolillo merged commit 1d07015 into main Jun 6, 2025
5 checks passed
@andreaangiolillo andreaangiolillo deleted the CLOUDP-311380 branch June 6, 2025 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cveticm cveticm cveticm approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@andreaangiolillo @cveticm